Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lsp: Revert URL type change #13193

Merged
merged 2 commits into from
Jun 18, 2024
Merged

lsp: Revert URL type change #13193

merged 2 commits into from
Jun 18, 2024

Conversation

osiewicz
Copy link
Contributor

@osiewicz osiewicz commented Jun 18, 2024

This reverts URI changes made in #12928 while keeping the perf goodies in tact. We should keep an eye out for gluon-lang/lsp-types#284
Fixes: #13135
Fixes: #13131
Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jun 18, 2024
@osiewicz osiewicz changed the title Lsp types revert url change lsp: Revert URL type change Jun 18, 2024
@osiewicz osiewicz merged commit 3a26a48 into main Jun 18, 2024
12 checks passed
@osiewicz osiewicz deleted the lsp-types-revert-url-change branch June 18, 2024 10:39
@tooltitude-support
Copy link

@osiewicz May I ask you share your experience in gluon-lang/lsp-types#284 so that people who try to migrate to the newer version of lsp-types have more information about the problems you encountered.

@osiewicz
Copy link
Contributor Author

osiewicz commented Jun 19, 2024

I don't think I have anything to add on that matter, sorry. I think upvoting the issue and referencing it in my PR should be sufficient, as bumping it is unlikely to push it forward any further.
Previous implementation of lsp_types::Url had a bunch of helpers which turned out to be quite handy in retrospect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
2 participants